Skip to content

Conversation

@jacobtylerwalls
Copy link
Member

@jacobtylerwalls jacobtylerwalls commented Jul 7, 2024

Type of Changes

Type
βœ“ πŸ”¨ Refactoring

Description

In preparation for dropping 3.8 support on main (#9774), clean up some items that were missed in previous rounds.

@jacobtylerwalls jacobtylerwalls added Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news πŸ”‡ This change does not require a changelog entry labels Jul 7, 2024
@jacobtylerwalls jacobtylerwalls requested a review from DudeNr33 as a code owner July 7, 2024 18:23
@@ -1,3 +1,3 @@
[testoptions]
max_pyver=3.8
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added in #9034, but I think the idea was to run this test on 3.8, not skip it on 3.8.

max_pyver is actually pretty confusingly named.

@codecov
Copy link

codecov bot commented Jul 8, 2024

Codecov Report

All modified and coverable lines are covered by tests βœ…

Project coverage is 95.84%. Comparing base (52955ba) to head (9e6e529).
Report is 143 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #9773   +/-   ##
=======================================
  Coverage   95.84%   95.84%           
=======================================
  Files         174      174           
  Lines       18878    18878           
=======================================
  Hits        18094    18094           
  Misses        784      784           

@DanielNoord DanielNoord merged commit 2c7a857 into main Jul 8, 2024
@DanielNoord DanielNoord deleted the cleanup-test-skips branch July 8, 2024 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news πŸ”‡ This change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants